-
Notifications
You must be signed in to change notification settings - Fork 9.8k
Add support for EKS Capability #45326
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Pankaj Walke <[email protected]>
|
✅ Thank you for correcting the previously detected issues! The maintainers appreciate your efforts to make the review process as smooth as possible. |
Community GuidelinesThis comment is added to every new Pull Request to provide quick reference to how the Terraform AWS Provider is maintained. Please review the information below, and thank you for contributing to the community that keeps the provider thriving! 🚀 Voting for Prioritization
Pull Request Authors
|
% make testacc TESTARGS='-run=TestAccEKSCapability_' PKG=eks make: Verifying source code with gofmt... ==> Checking that code complies with gofmt requirements... make: Running acceptance tests on branch: 🌿 HEAD 🌿... TF_ACC=1 go1.24.10 test ./internal/service/eks/... -v -count 1 -parallel 20 -run=TestAccEKSCapability_ -timeout 360m -vet=off 2025/12/01 15:00:37 Creating Terraform AWS Provider (SDKv2-style)... 2025/12/01 15:00:37 Initializing Terraform AWS Provider (SDKv2-style)... === RUN TestAccEKSCapability_basic === PAUSE TestAccEKSCapability_basic === RUN TestAccEKSCapability_disappears === PAUSE TestAccEKSCapability_disappears === RUN TestAccEKSCapability_tags === PAUSE TestAccEKSCapability_tags === CONT TestAccEKSCapability_basic === CONT TestAccEKSCapability_tags === CONT TestAccEKSCapability_disappears --- PASS: TestAccEKSCapability_disappears (730.41s) --- PASS: TestAccEKSCapability_tags (736.61s) --- PASS: TestAccEKSCapability_basic (755.84s) PASS ok github.com/hashicorp/terraform-provider-aws/internal/service/eks 761.264s
|
✅ Thank you for correcting the previously detected issues! The maintainers appreciate your efforts to make the review process as smooth as possible. |
% make testacc TESTARGS='-run=TestAccEKSCapability_argoCD' PKG=eks make: Verifying source code with gofmt... ==> Checking that code complies with gofmt requirements... make: Running acceptance tests on branch: 🌿 HEAD 🌿... TF_ACC=1 go1.24.10 test ./internal/service/eks/... -v -count 1 -parallel 20 -run=TestAccEKSCapability_argoCD -timeout 360m -vet=off 2025/12/01 17:12:27 Creating Terraform AWS Provider (SDKv2-style)... 2025/12/01 17:12:27 Initializing Terraform AWS Provider (SDKv2-style)... === RUN TestAccEKSCapability_argoCD === PAUSE TestAccEKSCapability_argoCD === CONT TestAccEKSCapability_argoCD --- PASS: TestAccEKSCapability_argoCD (914.69s) PASS ok github.com/hashicorp/terraform-provider-aws/internal/service/eks 920.195s
This is a valid use of `append` without re-assigning the result to the first argument.
…nv var
Test still failing, but no longer due to server side validation of the IDC group identifier.
```console
% WS_IDENTITY_STORE_USER_ID=c4b86468-0061-702e-f540-d17f8b3bc6f1 AWS_IDENTITY_STORE_GROUP_NAME=f48884f8-30e1-7057-0c44-848c0aca4ed2 make t K=eks T=TestAccEKSCapability_ArgoCD_rbac
make: Verifying source code with gofmt...
==> Checking that code complies with gofmt requirements...
make: Running acceptance tests on branch: 🌿 f-aws_eks_capability 🌿...
TF_ACC=1 go1.24.10 test ./internal/service/eks/... -v -count 1 -parallel 20 -run='TestAccEKSCapability_ArgoCD_rbac' -timeout 360m -vet=off
2025/12/02 13:47:51 Creating Terraform AWS Provider (SDKv2-style)...
2025/12/02 13:47:51 Initializing Terraform AWS Provider (SDKv2-style)...
=== RUN TestAccEKSCapability_ArgoCD_rbac
=== PAUSE TestAccEKSCapability_ArgoCD_rbac
=== CONT TestAccEKSCapability_ArgoCD_rbac
capability_test.go:217: Step 3/3 error: Error running apply: exit status 1
Error: Invalid list/set
with aws_eks_capability.test,
on terraform_plugin_test.tf line 119, in resource "aws_eks_capability" "test":
119: resource "aws_eks_capability" "test" {
too many elements: want 1, got 2
--- FAIL: TestAccEKSCapability_ArgoCD_rbac (1104.26s)
FAIL
FAIL github.com/hashicorp/terraform-provider-aws/internal/service/eks 1110.774s
```
ewbankkit
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀.
% AWS_IDENTITY_STORE_USER_ID=926776f3c6-53ea24ec-f8f8-4e7c-a015-d238f4ec12fb AWS_IDENTITY_STORE_GROUP_ID=926776f3c6-71733eb8-b181-488e-8ed6-17b07e79380e make testacc TESTARGS='-run=TestAccEKSCapability_ArgoCD_' PKG=eks
make: Verifying source code with gofmt...
==> Checking that code complies with gofmt requirements...
make: Running acceptance tests on branch: 🌿 HEAD 🌿...
TF_ACC=1 go1.24.10 test ./internal/service/eks/... -v -count 1 -parallel 20 -run=TestAccEKSCapability_ArgoCD_ -timeout 360m -vet=off
2025/12/02 16:21:53 Creating Terraform AWS Provider (SDKv2-style)...
2025/12/02 16:21:53 Initializing Terraform AWS Provider (SDKv2-style)...
=== RUN TestAccEKSCapability_ArgoCD_basic
=== PAUSE TestAccEKSCapability_ArgoCD_basic
=== RUN TestAccEKSCapability_ArgoCD_rbac
=== PAUSE TestAccEKSCapability_ArgoCD_rbac
=== CONT TestAccEKSCapability_ArgoCD_basic
=== CONT TestAccEKSCapability_ArgoCD_rbac
--- PASS: TestAccEKSCapability_ArgoCD_rbac (1021.30s)
--- PASS: TestAccEKSCapability_ArgoCD_basic (1114.91s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/eks 1121.572sCo-authored-by: Jared Baker <[email protected]>
Rollback Plan
If a change needs to be reverted, we will publish an updated version of the library.
Changes to Security Controls
Are there any changes to security controls (access controls, encryption, logging) in this pull request? If so, explain.
Description
Adds support for managing EKS Capabilities - https://docs.aws.amazon.com/eks/latest/userguide/capabilities.html
Relations
Closes #45324
References
Output from Acceptance Testing